Skip to content

[codex] add code review caller workflow#4

Merged
xdanger merged 2 commits into
mainfrom
xdanger-codex/code-review-workflow-v1
Apr 2, 2026
Merged

[codex] add code review caller workflow#4
xdanger merged 2 commits into
mainfrom
xdanger-codex/code-review-workflow-v1

Conversation

@xdanger
Copy link
Copy Markdown
Member

@xdanger xdanger commented Apr 2, 2026

Summary

  • add a pull-request caller workflow for shared code review

  • pin the reusable workflow reference to v1 for stability

Validation

  • git diff --check

xdanger added 2 commits March 30, 2026 14:08
- 🔧 avoid unreviewed changes from a floating workflow ref
@xdanger xdanger marked this pull request as ready for review April 2, 2026 10:59
@xdanger xdanger requested a review from Copilot April 2, 2026 10:59
@xdanger xdanger self-assigned this Apr 2, 2026
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a repository-level GitHub Actions workflow to automatically invoke a shared/reusable “code review” workflow on pull request events, with conditions to avoid running on drafts and forked PRs.

Changes:

  • Introduces .github/workflows/code-review.yml triggered on key pull_request activity types.
  • Calls the reusable workflow taptap/.github/.github/workflows/code-review.yml@v1 and forwards ANTHROPIC_API_KEY.
  • Sets workflow permissions for actions/content read, OIDC token write, and issue/PR write.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/code-review.yml
@xdanger
Copy link
Copy Markdown
Member Author

xdanger commented Apr 2, 2026

@greptileai review

@xdanger xdanger merged commit bc17cec into main Apr 2, 2026
6 checks passed
@xdanger xdanger deleted the xdanger-codex/code-review-workflow-v1 branch April 2, 2026 11:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants